Skip to content

Comments

feat(rel): change build names for single image#130

Merged
jdpleiness merged 1 commit intomainfrom
jdp/rel/change-build-names-for-burstable-QOS-prep
Dec 18, 2024
Merged

feat(rel): change build names for single image#130
jdpleiness merged 1 commit intomainfrom
jdp/rel/change-build-names-for-burstable-QOS-prep

Conversation

@jdpleiness
Copy link
Contributor

Images are no longer sized so remove duplicate builds and remove image size in description and name

Images are no longer sized so remove duplicate builds and remove image
size in description and name
@jdpleiness jdpleiness requested a review from a team December 18, 2024 20:05
@jdpleiness jdpleiness enabled auto-merge (squash) December 18, 2024 20:06
@@ -392,10 +392,10 @@ build {
name = "sourcegraph-amis"
sources = var.dev ? ["source.amazon-ebs.DEV"] : [
"source.amazon-ebs.XS",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't we get rid of this one too?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh wait, that's the only one left that doesn't need sizing anymore. I see

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I'll go in here and do some cleanup after. This is just getting it out the door for today.

@jdpleiness jdpleiness merged commit c277d4c into main Dec 18, 2024
@jdpleiness jdpleiness deleted the jdp/rel/change-build-names-for-burstable-QOS-prep branch December 18, 2024 20:10
DaedalusG added a commit that referenced this pull request Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants